Skip to content

Conversation

@kiminkim724
Copy link
Collaborator

Summary

To help #1768 work properly

  • Moves all the profile setting calls to one place so that onCreate, the code will have all the details it needs to populate the email.
  • Tested with the new functions in place, and the emails are populating the details correctly.

Checklist

  • On the frontend, I've made my strings translate-able.
  • If I've added shared components, I've added a storybook story.
  • I've made pages responsive and look good on mobile.

Screenshots

Known issues

  • Don't know if this might break with the current firebase objects, so we'll have to see

Steps to test/reproduce

  1. Create an organization profile
  2. See if the data is populated correctly.

@vercel
Copy link

vercel bot commented May 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
maple-dev Ready Ready Preview, Comment Jan 14, 2026 1:53am
maple-prod Building Building Preview, Comment Jan 14, 2026 1:53am

Copy link
Collaborator

@Mephistic Mephistic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks for getting this back in sync ahead of the SMS Verification work!

@Mephistic Mephistic merged commit 8f86d00 into codeforboston:main Jan 14, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants